Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Folder reconfig, adding reading data notebook, and Onboarding 4 notebook #38

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Folder reconfig, adding reading data notebook, and Onboarding 4 notebook #38

wants to merge 1 commit into from

Conversation

mbforr
Copy link
Contributor

@mbforr mbforr commented Mar 7, 2025

  1. Adding a new folder structure and removing old notebooks that are not working ATM or need review
.
├── README.md
├── advanced                
├── datasets
│   └── foursquare-places-example
│       └── assets
├── get-started
│   └── map-config
├── snippets
├── wherobots-ai
│   ├── conf
│   ├── gpu
│   │   ├── img
  1. New snippets section with code snippets and one new notebook
  2. New Onboarding Part 4 notebook

@gitnotebooks GitNotebooks
Copy link

gitnotebooks bot commented Mar 7, 2025

Found 17 changed notebooks. Review the changes at https://app.gitnotebooks.com/wherobots/wherobots-examples/pull/38

prantogg
prantogg approved these changes Mar 7, 2025
View reviewed changes
Copy link
Contributor

@prantogg prantogg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@RoboDonut
Copy link
Contributor

RoboDonut commented Mar 9, 2025 •
edited
Loading

We really shouldn't be storing datasets here. IIRC those assets were more images and configs and just needs a rename.

I was hoping we could look at a much more of a flat structure and consistency in the naming.

Something like
FeatureType_feature_lang e.g.

  • WherobotsDB-Importing-Python
  • WherobotsAI-DBScan-Scala
  • WherobotsAI-ObjectDetection-Python

When there is a hard requirement for a GPU or other org upgrade we highlight that the top of the first cell.

@RoboDonut
Copy link
Contributor

something more like this with consistent naming Product-Feature-Language. Asset names match the notebook with asset name suffix. A dir for on-boarding and a dir for partner notebooks.

cc @jiayuasu @james-willis @rbavery

├── assets/
│   ├── Foursquare-Places-Wherobots-map-config.json
│   ├── Foursquare-Places-Wherobots-choropleth.png
│   ├── Foursquare-Places-Wherobots-placemaker-tools-2.png
│   ├── Foursquare-Places-Wherobots-1.png
│   ├── WherobotsAI-MapConfig-JSON.json
│   ├── Wherobots-Onboarding-CentralPark-Config.json
│   ├── Wherobots-Onboarding-Config.json
│
├── notebooks/
│   ├── First-Wherobots-Notebook.ipynb
│   ├── WherobotsDB-KNN-Python.ipynb
│   ├── WherobotsDB-OvertureMaps-Python.ipynb
│   ├── WherobotsDB-STACReader-Python.ipynb
│   ├── WherobotsDB-TileGeneration-Python.ipynb
│   ├── WherobotsDB-Example-Python.ipynb
│   ├── WherobotsAI-DBSCAN-Python.ipynb
│   ├── WherobotsAI-Glocal-Python.ipynb
│   ├── WherobotsAI-LOF-Python.ipynb
│   ├── WherobotsAI-MapMatching-Python.ipynb
│   ├── WherobotsAI-ObjectDetection-Python.ipynb
│   ├── WherobotsAI-Segmentation-Python.ipynb
│   ├── WherobotsAI-GPU-BringYourOwnModel-Python.ipynb
│   ├── WherobotsAI-GPU-Classification-Python.ipynb
│   ├── WherobotsAI-GPU-ObjectDetection-Python.ipynb
│   ├── WherobotsAI-GPU-Segmentation-Python.ipynb
│   ├── WherobotsDB-Havasu-Iceberg-Geometry-ETL-Python.ipynb
│   ├── WherobotsDB-Havasu-Iceberg-OutDB-Raster-ETL-Python.ipynb
│   ├── WherobotsDB-Havasu-Iceberg-Raster-ETL-Python.ipynb
│   ├── ESA-WorldCover-Python.ipynb
│   ├── WherobotsDB-KNN-Scala.ipynb
│   ├── WherobotsDB-TileGeneration-Scala.ipynb
│   ├── WherobotsDB-Example-Scala.ipynb
│   ├── WherobotsDB-Havasu-Iceberg-Geometry-ETL-Scala.ipynb
│
├── onboarding/
│   ├── Wherobots-Onboarding-Part1-LoadingData.ipynb
│   ├── Wherobots-Onboarding-Part2-LoadingData.ipynb
│   ├── Wherobots-Onboarding-Part3-Accelerating-Geospatial-Datasets.ipynb
│
├── partner_examples/
│   ├── Foursquare-Places-Wherobots-Python.ipynb
│
├── sedona-maven-example/
│   ├── .gitignore
│   ├── pom.xml
│   ├── src/
│   │   ├── main/
│   │   │   ├── scala/
│   │   │   │   ├── com/
│   │   │   │   │   ├── wherobots/
│   │   │   │   │   │   ├── sedona/
│   │   │   │   │   │   │   ├── SedonaDbExample.scala
│   │   ├── test/
│   │   │   ├── resources/
│   │   │   │   ├── .gitignore
│   │   │   │   ├── scalastyle_config.xml
│
├── README.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@prantogg prantogg prantogg approved these changes

@mpetazzoni mpetazzoni Awaiting requested review from mpetazzoni

@RoboDonut RoboDonut Awaiting requested review from RoboDonut

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mbforr @RoboDonut @prantogg